make some Qt containers const. (#705)
authortsteven4 <13596209+tsteven4@users.noreply.github.com>
Mon, 15 Mar 2021 20:32:54 +0000 (14:32 -0600)
committerGitHub <noreply@github.com>
Mon, 15 Mar 2021 20:32:54 +0000 (14:32 -0600)
This avoids checks for detachment.

mynav.cc
pcx.cc
shape.cc
xcsv.cc

index 95a614ac20b5d926b2e4b8f18e929da7c9fc41ae..9643daeea29a7ed02db1c6537c5eaa087f5aebb3 100644 (file)
--- a/mynav.cc
+++ b/mynav.cc
@@ -41,7 +41,7 @@
 void
 MyNavFormat::read_line(const QString& line, route_head* track)
 {
-  QStringList fields = line.split("|");
+  const QStringList fields = line.split('|');
 
   if (global_opts.debug_level > 1) {
     qDebug() << "line: " << line;
diff --git a/pcx.cc b/pcx.cc
index 8b53970c7175680573362177a489423f45b32eda..e9658d7f1a7d4bc872348961fe1f947c4a488ed0 100644 (file)
--- a/pcx.cc
+++ b/pcx.cc
@@ -132,8 +132,7 @@ static void data_read() {
 
     switch (ibuf[0]) {
       case 'W': {
-        QStringList tokens =
-            line.split(sep);
+        const QStringList tokens = line.split(sep);
         if (tokens.size() < 6) {
           fatal(MYNAME
                 ": Unable to parse waypoint, not all required columns "
@@ -232,8 +231,7 @@ static void data_read() {
         route_add_head(route);
         break;
       case 'T': {
-        QStringList tokens =
-            line.split(sep);
+        const QStringList tokens = line.split(sep);
         if (tokens.size() < 6) {
           fatal(MYNAME
                 ": Unable to parse trackpoint, not all required columns "
index 3fe7ba6706f1e1551d7aa79d2e7e089268fb0f5a..1c732153c933504beed686c4b592f3007ff142e6 100644 (file)
--- a/shape.cc
+++ b/shape.cc
@@ -218,7 +218,7 @@ ShapeFormat::read()
     if (qopt_name.contains('+')) {
       // form a compound name from one or more fields.
       nameidx = -2;
-      QStringList opt_name_fields = qopt_name.split('+', QString::SkipEmptyParts);
+      const QStringList opt_name_fields = qopt_name.split('+', QString::SkipEmptyParts);
       nameindices.reserve(opt_name_fields.size());
       for (int oidx=0; oidx<opt_name_fields.size(); oidx++) {
         bool ok;
diff --git a/xcsv.cc b/xcsv.cc
index 47b33455a6d29ca97c31059af654a45aadce3588..4bb7723fa39956ca3f1e8e824b29a94e100d81f4 100644 (file)
--- a/xcsv.cc
+++ b/xcsv.cc
@@ -1657,7 +1657,7 @@ XcsvStyle::xcsv_parse_style_line(XcsvStyle* style, QString line)
   // the first token is the operation, e.g. "IFIELD"
   QString op = line.mid(0, sep).trimmed().toUpper();
   QString tokenstr = line.mid(sep).trimmed();
-  QStringList tokens = tokenstr.split(",");
+  const QStringList tokens = tokenstr.split(',');
 
   if (op == "FIELD_DELIMITER") {
     auto cp = xcsv_get_char_from_constant_table(tokens[0]);